Skip to content

Support MariaDB JDBC driver - #7912

Merged
labkey-adam merged 7 commits into
release26.3-SNAPSHOTfrom
26.3_fb_mariadb
Aug 14, 2026
Merged

Support MariaDB JDBC driver#7912
labkey-adam merged 7 commits into
release26.3-SNAPSHOTfrom
26.3_fb_mariadb

Conversation

@labkey-adam

@labkey-adam labkey-adam commented Aug 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Rationale

Supporting MariaDB means more keyword candidates

Related Pull Requests

Changes

  • Add some MariaDB-specific keywords
  • Test mysql.sakila database if it's present
  • Mask password parameter values when logging the connection URL
  • Stricter JDBC URL parsing to support MariaDB no database + no slash option

@labkey-adam
labkey-adam requested a review from a team August 10, 2026 22:57
@labkey-tchad

labkey-tchad commented Aug 13, 2026

Copy link
Copy Markdown
Member

Manual testing looks good. I updated the TableSelectorTestCase to work without having to comment it out when there's no 'mySql' data source.
Looks like the URL parsing change introduced a failure to OracleDialectFactory$JdbcHelperTestCase

@labkey-adam

labkey-adam commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

Manual testing looks good. I updated the TableSelectorTestCase to work without having to comment it out when there's no 'mySql' data source. Looks like the URL parsing change introduced a failure to OracleDialectFactory$JdbcHelperTestCase

Thanks for flagging that failure

@labkey-adam
labkey-adam merged commit ba3ae3c into release26.3-SNAPSHOT Aug 14, 2026
8 checks passed
@labkey-adam
labkey-adam deleted the 26.3_fb_mariadb branch August 14, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@labkey-jeckels labkey-jeckels labkey-jeckels approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@labkey-adam @labkey-tchad @labkey-jeckels